Skip to content

[Code Update - Chapter 09] Code update for MP Telemetry 2.1#72

Open
ttelang wants to merge 15 commits into
microprofile:mainfrom
ttelang:code-update-mp-telemetry-2.1
Open

[Code Update - Chapter 09] Code update for MP Telemetry 2.1#72
ttelang wants to merge 15 commits into
microprofile:mainfrom
ttelang:code-update-mp-telemetry-2.1

Conversation

@ttelang
Copy link
Copy Markdown
Contributor

@ttelang ttelang commented Apr 24, 2026

Below are the list of changes:

pom.xml

  • Updating dependencies for Fault Tolerance 4.1 (MicroProfile 7.1)
  • Migrating to Java 21

README.adoc

  • Update README.adoc as per the MP Telemetry 2.1 implementation
  • Removing content already covered in previous chapters.

server.xml

  • updating dependencies

PaymentResource.java

  • updating PaymentResource implementation with checkGatewayHealth() and asynchronous sendNotification() method.

micrprofile-config.properties

  • Update MicroProfile Telemetry Configuration

index.html

  • Update version numbers

ttelang added 9 commits March 27, 2026 17:15
Updated sections on telemetry data exporting, metrics, and logs. Added new content for MicroProfile Telemetry 2.1 features.
- Clarified OpenTelemetry and MicroProfile Telemetry Integration.
- Fixing grammatical & styling issues to ensure adherence to Eclipse Foundation Writing Style Guide throughout this chapter.
Update README.adoc as per the mp telemetry 2.1 implementation
@ttelang ttelang marked this pull request as ready for review April 24, 2026 17:34
@ttelang ttelang changed the title [Code Update - Chapter 09 - MP Telemetry 4.1] Code update for MP Telemetry 2.1 [Code Update - Chapter 09] Code update for MP Telemetry 2.1 Apr 24, 2026
Comment thread code/chapter09/payment/README.adoc Outdated
ttelang and others added 6 commits May 17, 2026 11:27
- Add docker-compose.yml with full LGTM stack (Grafana, Loki, Tempo, Prometheus, OTel Collector)
- Add otel-collector-config.yml with correct OTLP exporter syntax and Prometheus port (8889)
- Add prometheus.yml configured to scrape OTel Collector metrics endpoint
- Enable logs exporter (otlp) and add Codespaces port-forward note in microprofile-config.properties

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ethods

- Switch Tracer from GlobalOpenTelemetry.getTracer() to CDI @Inject per MP Telemetry 2.1 spec
- Add CDI-injected Meter with paymentAttemptsCounter tracking success/failed/fallback outcomes
- Add missing checkGatewayHealth() with @CIRCUITBREAKER called by PaymentResource
- Add missing sendPaymentNotification() with @asynchronous @bulkhead called by PaymentResource
- Remove type pom from microprofile-telemetry-api dependency, it is a JAR not a BOM
- Enable otel.logs.exporter=otlp in README, was incorrectly set to none in two places
- Replace all Jaeger and Zipkin references in README with LGTM stack
- Add Grafana/Tempo trace verification steps (port 13000)
- Add Loki log correlation steps with trace-to-log linking
- Add Prometheus metrics verification at port 19090
- Replace cleanup step (docker rm jaeger) with docker-compose down
- Remove microprofile-telemetry-api JAR dependency from pom.xml;
  the artifact is only published as a POM on Maven Central, not a JAR.
  opentelemetry-api:1.48.0:provided covers all OTel types at compile time.
- Added `dns_search: []` to all services in `docker-compose.yml` to prevent DNS resolution issues in GitHub Codespaces.
- Updated `otel-collector-config.yml` to use new OTLP endpoints for Tempo and Loki.
- Introduced `tempo-config.yaml` for Grafana Tempo v3 compatibility.
- Updated `README.adoc` to reflect changes in architecture and service descriptions.
- Added images for Grafana dashboards related to payment service telemetry.
- Changed OTLP endpoint in `microprofile-config.properties` to accommodate GitHub Codespaces.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants